Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for multiple pin codes #30

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lalitindoria
Copy link

No description provided.

abhilashkulkarniofficial and others added 3 commits May 2, 2021 14:56
* Changed email body

Added html email template to the email body to display the slots information in a cleaner template.

* Removed bug in creating template

* Updated Readme to add new email template
Revert was done as createTemplate method iterates on a string, and throws errors.

This reverts commit f04624d.
@Soniya318
Copy link

is it working?

@lalitindoria
Copy link
Author

is it working?

yes, works for me.

@RaviPabari
Copy link

have you received any mails if slot is available?

@lalitindoria
Copy link
Author

lalitindoria commented May 8, 2021 via email

@RaviPabari
Copy link

No slots are available for 18+ so didn't get any. Got emails when I changed
age to 45+.

On Fri, May 7, 2021 at 9:27 PM Ravi Pabari @.***> wrote:

have you received any mails if slot is available?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#30 (comment),
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYBC7Y5UL56RX2LRTWDBS3TMQEXNANCNFSM44AJCP3A
.

--
Lalit Indoria

And now the API will have cached data so can't rely on this anymore :/

@lalitindoria
Copy link
Author

lalitindoria commented May 10, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants